Skip to content

docs(pre-requisites): change the RouterModule addition#539

Merged
SanderElias merged 1 commit intoscullyio:masterfrom
geromegrignon:master
May 13, 2020
Merged

docs(pre-requisites): change the RouterModule addition#539
SanderElias merged 1 commit intoscullyio:masterfrom
geromegrignon:master

Conversation

@geromegrignon
Copy link
Copy Markdown
Contributor

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Other... Please describe: doc update

What is the current behavior?

The schematic listed into the documentation ng generate module app-routing --flat --module=app adds a module without RouterModule

What is the new behavior?

  • move up the warning about RouterModule pre-requisite
  • remove the schematic command adding a routing module
  • add --routing to the existing creation commands

Close #534

Does this PR introduce a breaking change?

  • Yes
  • No

- move up the warning about routermodule pre-requisites.md
- remove the schematci command to add a routing module
- add --routing to the existing creation commands
@SanderElias SanderElias merged commit 54ab8c5 into scullyio:master May 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Couldn't find a route declaration in src/app/app-routing.module.ts

2 participants